Skip to content

Comments

fix: fix incorrectly provided error object to pino#539

Merged
snario merged 2 commits intomasterfrom
liam/pino-error
Apr 21, 2021
Merged

fix: fix incorrectly provided error object to pino#539
snario merged 2 commits intomasterfrom
liam/pino-error

Conversation

@snario
Copy link
Contributor

@snario snario commented Apr 21, 2021

Error must be provided inside of an err object.

@changeset-bot
Copy link

changeset-bot bot commented Apr 21, 2021

🦋 Changeset detected

Latest commit: 5563bcd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/batch-submitter Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@snario snario merged commit c7bc0ce into master Apr 21, 2021
@snario snario deleted the liam/pino-error branch April 21, 2021 19:25
@snario snario restored the liam/pino-error branch April 21, 2021 19:25
@snario snario deleted the liam/pino-error branch April 21, 2021 19:25
theochap pushed a commit that referenced this pull request Dec 10, 2025
* fix(client): Channel reader error handling

* fmt
theochap pushed a commit that referenced this pull request Jan 15, 2026
### Description

A thin wrapper around `OpExecutionPayload` that includes the parent
beacon block root that provides a rust-equivalent to
[`ExecutionPayloadEnvelope`](https://github.com/ethereum-optimism/optimism/blob/7c812674f9e2068949a152f44fde13fcb93b2c45/op-service/eth/types.go#L233).

This is required for a serialized rpc type in the op admin api to post
unsafe payloads.
emhane added a commit that referenced this pull request Feb 4, 2026
Fix broken unit test for verifying `Debug` impl, add missing map
`address_mappings` to expected test output, introduced in
op-rs/op-reth#512
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants